-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
restore the line to initialize 'raterr2' as zero #668
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xyzemc , Thanks taking care of this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for spotting this.
Thank you @TingLei-NOAA and @emilyhcliu for your quick reviews. @xyzemc , now that we have two peer review approvals, we need to run regression (ctests) on WCOSS2, Hera, and Orion. Would you please do so? |
Yes, I will run the regression test. |
Oops, @xyzemc , when I said post the ctest results I was only referring to the ctest output. For example, look in PR #669. Gang cut-n-pasted the ctest output under the How Has This Been Tested? section of the PR. Click here to see a different example from PR #654. |
The regression tests on Hera passed: Test project /scratch1/NCEPDEV/stmp2/Xiaoyan.Zhang/regrssion/develop/build 1/2 Test #1: [=[global_4denvar]=] ............. Passed 4016.24 sec 100% tests passed, 0 tests failed out of 2 Total Test time (real) = 6354.41 sec |
@xyzemc , why are only the global ctests being run ( |
Thanks! I will do all the ctest on Hera, wcoss, and orion today. |
The regression tests on wcoss2 passed: 100% tests passed, 0 tests failed out of 7 Total Test time (real) = 1493.65 sec |
The regression tests on Hera passed: 100% tests passed, 0 tests failed out of 7 Total Test time (real) = 3000.60 sec |
The regression tests on Orion: 43% tests passed, 4 tests failed out of 7 Total Test time (real) = 1817.56 sec The following tests FAILED:
Output from these tests are in: /work/noaa/fv3-cam/xyzhang/tmpreg_global_enkf/tmpreg_global_enkf/global_enkf_loproc_updat_vs_global_enkf_loproc_contrl/global_enkf_regression_results.txt The Test 4 netcdf_fv3_regional failed due to a runtime check
The test results is in: /work/noaa/fv3-cam/xyzhang/tmpreg_netcdf_fv3_regional/tmpreg_netcdf_fv3_regional/netcdf_fv3_regional_loproc_updat_vs_netcdf_fv3_regional_loproc_contrl/netcdf_fv3_regional_regression_results.txt The Test 3 rrfs_3denvar_glbens failed due to a runtime check
The test results is in: /tmpreg_rrfs_3denvar_glbens/rrfs_3denvar_glbens_loproc_updat_vs_rrfs_3denvar_glbens_loproc_contrl/rrfs_3denvar_glbens_regression_results.txt The Test 5 rrfs_3denvar_glbens failed due to a runtime check
The test results is in: /work/noaa/fv3-cam/xyzhang/tmpreg_rrfs_3denvar_glbens/tmpreg_rrfs_3denvar_glbens/rrfs_3denvar_glbens_loproc_updat_vs_rrfs_3denvar_glbens_loproc_contrl/rrfs_3denvar_glbens_regression_results.txt |
Thank you @xyzemc for running regression tests on WCOSS2, Hera, and Orion. The Orion failures are not fatal. Given the two peer approvals and documented ctests, I will forward this PR to the Handling Review Team for action. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Simple PR which ensures raterr2
is not uninitialized.
Approve.
This PR is to fix the issue #648
Fixes #648
DUE DATE for merger of this PR into
develop
is 6/29/2023 (six weeks after PR creation).